Skip to content

MockFile.AppendAllText creates the file if it doesn't exist.#55

Merged
tathamoddie merged 1 commit intoTestableIO:masterfrom
wtjones:AppendAllText-create
Dec 1, 2013
Merged

MockFile.AppendAllText creates the file if it doesn't exist.#55
tathamoddie merged 1 commit intoTestableIO:masterfrom
wtjones:AppendAllText-create

Conversation

@wtjones
Copy link
Copy Markdown
Contributor

@wtjones wtjones commented Nov 23, 2013

I noticed this issue when mocking in another project.

@wtjones
Copy link
Copy Markdown
Contributor Author

wtjones commented Dec 1, 2013

To clarify, the issue is that the file isn't created. This patch corrects that behavior.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This extra file seems extraneous for the test?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, never mind. It's because you need the folder...

tathamoddie added a commit that referenced this pull request Dec 1, 2013
MockFile.AppendAllText creates the file if it doesn't exist.
@tathamoddie tathamoddie merged commit 8dffd3c into TestableIO:master Dec 1, 2013
@tathamoddie
Copy link
Copy Markdown
Contributor

Thanks. The build server will push this to NuGet within the next few minutes.

@wtjones
Copy link
Copy Markdown
Contributor Author

wtjones commented Dec 2, 2013

Yea I wanted to just add a folder for the test but wasn't sure how the
drive letter is initialized without a starting item.

Thanks.

On Sun, Dec 1, 2013 at 3:45 PM, Tatham Oddie notifications@github.comwrote:

Thanks. The build server will push this to NuGet within the next few
minutes.


Reply to this email directly or view it on GitHubhttps://github.com//pull/55#issuecomment-29584012
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants